hw4 #94
hw4 #94
Conversation
Rustyam
force-pushed
the
RMulyukov/hw4
branch
from
2ac2c34 to
ab25b39
Compare
app/app/Response.php
Outdated
|
|
||
| private function initSuccess(): void | ||
| { | ||
| header("HTTP/1.1 200 OK"); |
There was a problem hiding this comment.
Сайд эффект. Заголовок сразу выставится и потом мы не сможем поменять, что может привести к неожиданным проблемам.
| private function initSuccess(): void | ||
| { | ||
| header("HTTP/1.1 200 OK"); | ||
| $this->message = 'OK'; |
There was a problem hiding this comment.
Зачем нам тут устанавливать внутреннее значения сообщения?
There was a problem hiding this comment.
в задании написано
• если строка корректна, то пользователю возвращается ответ 200 OK, с информационным текстом, что всё хорошо;
• если строка некорректна, то пользователю возвращается ответ 400 Bad Request, с информационным текстом, что всё плохо.
$this->message содержит этот информационный текст
There was a problem hiding this comment.
Вопрос в том, что тут мы устанавливаем заголовок и записываем значение в message, чтобы потом его получить и напечатать. Не будет ли лучше сразу напечатать и всё?
No description provided.